Skip to content

Introduce Tracer API for log capture. - #3515

Merged
SylvainJuge merged 3 commits into
elastic:mainfrom
raphw:logging-api
Feb 9, 2024
Merged

Introduce Tracer API for log capture.#3515
SylvainJuge merged 3 commits into
elastic:mainfrom
raphw:logging-api

Conversation

@raphw

@raphw raphw commented Feb 5, 2024

Copy link
Copy Markdown
Contributor

This PR moves API for log reporting to the Tracer API rather then relying on the agent API implementation. As logs are only consumed, this API can trivially be implemented as non-operational.

@github-actions github-actions Bot added agent-java community Issues and PRs created by the community triage labels Feb 5, 2024
@github-actions

github-actions Bot commented Feb 5, 2024

Copy link
Copy Markdown

👋 @raphw Thanks a lot for your contribution!

It may take some time before we review a PR, so even if you don’t see activity for some time, it does not mean that we have forgotten about it.

Every once in a while we go through a process of prioritization, after which we are focussing on the tasks that were planned for the upcoming milestone. The prioritization status is typically reflected through the PR labels. It could be pending triage, a candidate for a future milestone, or have a target milestone set to it.

@SylvainJuge SylvainJuge left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot for simplifying this @raphw , from my side it looks great, I'll ask someone else in the team for another review on this.

@SylvainJuge SylvainJuge removed the triage label Feb 6, 2024
@SylvainJuge

Copy link
Copy Markdown
Member

@elasticmachine run elasticsearch-ci/docs

@v1v

v1v commented Feb 9, 2024

Copy link
Copy Markdown
Member

run docs-build

@SylvainJuge
SylvainJuge merged commit d178642 into elastic:main Feb 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agent-java community Issues and PRs created by the community

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants